Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support using the target agent puppet config #3255

Closed
wants to merge 1 commit into from

Conversation

h0tw1r3
Copy link
Contributor

@h0tw1r3 h0tw1r3 commented Dec 14, 2023

modules like puppetlabs-puppetdb which make use of Puppet.runtime[:http] require the agent ssl certificate and key. this change supports an optional flag which simply skips the confdir setting override when initializing the Puppet object.

@h0tw1r3 h0tw1r3 added Feature New features and improvements. Do Not Merge Work that should not be merged yet. Request for Comment More use case are needed to act on the ticket. labels Dec 14, 2023
@h0tw1r3 h0tw1r3 requested a review from a team as a code owner December 14, 2023 19:34
modules like puppetlabs-puppetdb which make use of Puppet.runtime[:http]
require the agent ssl certificate and key. this change supports an
optional flag which simply skips the confdir setting override when
initializing the Puppet object.
@donoghuc
Copy link
Contributor

Apply blocks intentionally isolate from existing agent settings. A generalized option like this I think will have many unintended consequences. Perhaps we can brainstorm some other way to pipe through specific settings for a use case like this.

h0tw1r3 added a commit to h0tw1r3/bolt that referenced this pull request Dec 15, 2023
!feature !experimental

* **add new option to apply-settings** ([puppetlabs#3255](puppetlabs#3255))

A new enum option 'confdir' supports overriding the default behavior
of using a bolt temporary confdir for each Puppet apply. This feature is
currently considered experimental.
@h0tw1r3 h0tw1r3 closed this Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge Work that should not be merged yet. Feature New features and improvements. Request for Comment More use case are needed to act on the ticket.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants